Skip to content

modern ircdocs tweak#120

Closed
AeliusSaionji wants to merge 1 commit intoadsr:masterfrom
AeliusSaionji:patch-2
Closed

modern ircdocs tweak#120
AeliusSaionji wants to merge 1 commit intoadsr:masterfrom
AeliusSaionji:patch-2

Conversation

@AeliusSaionji
Copy link
Copy Markdown

I'm in the process of requesting that modern-ircdocs be revised to avoid this situation, but in the event that this doesn't go through, here's the bare minimum of what fussy clients will expect.

I'm in the process of requesting that modern-ircdocs be revised to avoid this situation, but in the event that this doesn't go through, here's the bare minimum of what fussy clients will expect
// Send RPL_ISUPPORT
this.ircd.write(ircUser.socket, 'irslackd', '005', [
ircUser.ircNick,
'CHANTYPES=#',
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's also a debug channel that uses the & prefix. Would that also belong here?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes

this.ircd.write(ircUser.socket, 'irslackd', '005', [
ircUser.ircNick,
'CHANTYPES=#',
':are supported by this server'
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: no need to manually stick the :, the library will do it for you.

@adsr
Copy link
Copy Markdown
Owner

adsr commented Jul 27, 2025

Superseded by #130

@adsr adsr closed this Jul 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants